Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Reformat BUILD.gn files to comply with the format checker presubmit script#13401

Merged
jason-simmons merged 1 commit into
flutter-team-archive:masterfrom
jason-simmons:build_gn_format_102819
Oct 29, 2019
Merged

Reformat BUILD.gn files to comply with the format checker presubmit script#13401
jason-simmons merged 1 commit into
flutter-team-archive:masterfrom
jason-simmons:build_gn_format_102819

Conversation

@jason-simmons

Copy link
Copy Markdown
Contributor

No description provided.

@iskakaushik iskakaushik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chinmaygarde

Copy link
Copy Markdown
Contributor

How did this pass presubmits before?

@jason-simmons

Copy link
Copy Markdown
Contributor Author

The ci/check_gn_format.py script was intending to do a dry run of gn format that would return a result indicating whether the build files were correctly formatted. But instead it's doing a real run of gn format that modifies the files and doesn't return the result code.

I'll fix the check_gn_format.py script.

@jason-simmons jason-simmons merged commit 7069025 into flutter-team-archive:master Oct 29, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 29, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 29, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Oct 30, 2019
git@github.com:flutter/engine.git/compare/419f5d594afc...46c757b

git log 419f5d5..46c757b --no-merges --oneline
2019-10-29 iska.kaushik@gmail.com [dart_runner] Common libs need to exist for aot runner (flutter-team-archive/engine#13419)
2019-10-29 jason-simmons@users.noreply.github.com Fix the dry run mode of the GN format checker script (flutter-team-archive/engine#13406)
2019-10-29 jason-simmons@users.noreply.github.com Reformat BUILD.gn files to comply with the format checker presubmit script (flutter-team-archive/engine#13401)
2019-10-29 jason-simmons@users.noreply.github.com Use DartExecutor.getBinaryMessenger in FlutterNativeView instead of deprecated send methods (flutter-team-archive/engine#13403)
2019-10-29 bkonyi@google.com Roll src/third_party/dart e1fce75301..0b96f9f9ae (28 commits)
2019-10-29 matthew-carroll@users.noreply.github.com Converted ActivityAware and ServiceAware Lifecycles to opaque objects (#43670) (flutter-team-archive/engine#13402)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC cbracken@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
Inconnu08 pushed a commit to Inconnu08/flutter that referenced this pull request Nov 26, 2019
git@github.com:flutter/engine.git/compare/419f5d594afc...46c757b

git log 419f5d5..46c757b --no-merges --oneline
2019-10-29 iska.kaushik@gmail.com [dart_runner] Common libs need to exist for aot runner (flutter-team-archive/engine#13419)
2019-10-29 jason-simmons@users.noreply.github.com Fix the dry run mode of the GN format checker script (flutter-team-archive/engine#13406)
2019-10-29 jason-simmons@users.noreply.github.com Reformat BUILD.gn files to comply with the format checker presubmit script (flutter-team-archive/engine#13401)
2019-10-29 jason-simmons@users.noreply.github.com Use DartExecutor.getBinaryMessenger in FlutterNativeView instead of deprecated send methods (flutter-team-archive/engine#13403)
2019-10-29 bkonyi@google.com Roll src/third_party/dart e1fce75301..0b96f9f9ae (28 commits)
2019-10-29 matthew-carroll@users.noreply.github.com Converted ActivityAware and ServiceAware Lifecycles to opaque objects (flutter#43670) (flutter-team-archive/engine#13402)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC cbracken@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

4 participants